Skip to content

Update MIT xPRO unauthenticated navbar for upstream marketing-site removal#77

Closed
asadali145 with Copilot wants to merge 2 commits into
masterfrom
copilot/update-theme-templates
Closed

Update MIT xPRO unauthenticated navbar for upstream marketing-site removal#77
asadali145 with Copilot wants to merge 2 commits into
masterfrom
copilot/update-theme-templates

Conversation

Copilot AI commented Jul 13, 2026

Copy link
Copy Markdown

This updates the MIT xPRO theme to match the upstream removal of ENABLE_MKTG_SITE / MKTG_URL_LINK_MAP-based template gating, while preserving the existing MIT xPRO-specific navigation content. In this repo, the only affected override is the unauthenticated header navbar.

  • Navbar template

    • Removes the mktg_site_enabled assignment from lms/templates/header/navbar-not-authenticated.html
    • Removes the surrounding Mako conditional so MIT xPRO marketing links render unconditionally
    • Preserves all MIT xPRO-specific links and structure, including the existing courses_are_browsable guard on COURSES
  • Template cleanup

    • Drops unused reverse and text_type imports from the same template after removing the obsolete branch
  • Instructor email templates

    • Checked for theme overrides of:
      • lms/templates/instructor/edx_ace/addbetatester/email/body.html
      • lms/templates/instructor/edx_ace/allowedenroll/email/body.html
    • No matching files exist in this repository, so no email-template changes are included here
<div class="mobile-nav-item hidden-mobile nav-item">
  <a href="${marketing_link('HOW_IT_WORKS')}">${_("How it Works")}</a>
</div>
% if courses_are_browsable:
  <div class="mobile-nav-item hidden-mobile nav-item">
    <a href="${marketing_link('COURSES')}">${_("Courses")}</a>
  </div>
% endif
<div class="mobile-nav-item hidden-mobile nav-item">
  <a href="${marketing_link('SCHOOLS')}">${_("Schools")}</a>
</div>

Copilot AI changed the title [WIP] Update theme templates to match upstream changes Update MIT xPRO unauthenticated navbar for upstream marketing-site removal Jul 13, 2026
Copilot AI requested a review from asadali145 July 13, 2026 13:06
@asadali145 asadali145 closed this Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants